-
Notifications
You must be signed in to change notification settings - Fork 38.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Node E2E: Fix remote log fetching. #37338
Node E2E: Fix remote log fetching. #37338
Conversation
@@ -299,7 +299,7 @@ func RunRemote(archive string, host string, cleanup bool, junitFilePrefix string | |||
// journald nodes. We should have a more robust way to collect logs. | |||
var ( | |||
logName = "system.log" | |||
logPath = filepath.Join(workspace, logName) | |||
logPath = fmt.Sprintf("/tmp/%s-%s", getTimestamp(), logName) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if node is rebooted, the log will still be saved?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
talked offline, it will be saved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Jenkins GCI GCE e2e failed for commit e000ff0. Full PR test history. The magic incantation to run this job again is |
@k8s-bot gci gce e2e test this |
I see. You did change the path in #35868 :) |
Automatic merge from submit-queue |
For issue #37333.
This will help debug #37333.
Mark v1.5 because this helps debug an issue #37333, which was originally #35935. /cc @saad-ali
@yujuhong @dchen1107 @jingxu97
/cc @kubernetes/sig-node
This change is